Skip to content

Conversation

@Pouyanpi
Copy link
Collaborator

@Pouyanpi Pouyanpi commented Sep 27, 2024

To enable passthrough mode in Colang 2.x one needs to import passthrough module and activate llm continuation in the main flow.


import passthrough
flow main
  activate llm continuation
  ...
  
  • Change action name and flow names
  • resolve flow competitions
  • use raw_response contextvar
  • add tests

@Pouyanpi Pouyanpi requested a review from drazvan September 27, 2024 10:24
@Pouyanpi Pouyanpi self-assigned this Oct 1, 2024
@Pouyanpi Pouyanpi force-pushed the feat/passthrough-colang-2 branch from cb683dd to c3a42b7 Compare October 11, 2024 15:12
@schuellc-nvidia
Copy link
Collaborator

I am not sure whats the motivation for this feature. Could you elaborate? Thanks!

@Pouyanpi
Copy link
Collaborator Author

I am not sure whats the motivation for this feature. Could you elaborate? Thanks!

We have this feature in Colang 1.0 , so when passthrough is enabled the LLM is queried directly with the user message. So other tasks are not invoked and a prompt template is not used etc.

A request can goes directly to the LLM. We can discuss wether it should be part of Colang 2.x or not.

@Pouyanpi Pouyanpi force-pushed the feat/passthrough-colang-2 branch from b38b13e to ac9bcad Compare October 16, 2024 15:14
@Pouyanpi Pouyanpi force-pushed the feat/passthrough-colang-2 branch from ac9bcad to 6fad14b Compare October 17, 2024 08:30
@Pouyanpi Pouyanpi marked this pull request as ready for review October 17, 2024 08:30
@Pouyanpi
Copy link
Collaborator Author

@schuellc-nvidia feel free to review. Is the test OK with you?.

@Pouyanpi Pouyanpi force-pushed the feat/passthrough-colang-2 branch from a660054 to 5e5ea8f Compare October 18, 2024 15:36
Copy link
Collaborator

@schuellc-nvidia schuellc-nvidia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good

@Pouyanpi Pouyanpi merged commit 0f02fd1 into develop Oct 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants